Skip to content

gh-149044: Implement PEP 820 – PySlot: Unified slot system for the C API#149055

Open
encukou wants to merge 46 commits intopython:mainfrom
encukou:slots-pep820-mrg
Open

gh-149044: Implement PEP 820 – PySlot: Unified slot system for the C API#149055
encukou wants to merge 46 commits intopython:mainfrom
encukou:slots-pep820-mrg

Conversation

@encukou
Copy link
Copy Markdown
Member

@encukou encukou commented Apr 27, 2026

@encukou encukou added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 28, 2026
@bedevere-bot
Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @encukou for commit c8e9133 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F149055%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

Copy link
Copy Markdown
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation looks pretty good. I haven't looked at all of the C code yet.

Comment thread Doc/c-api/slots.rst Outdated
Comment on lines +148 to +149
that C++ added in the 2000 version of the standard.
If your code needs to be compatible with C++11 or older,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you mean "2020", not "2000". It might be better to say C++20 instead of "2020 version of the standard".

Comment thread Doc/c-api/slots.rst Outdated
@AlexWaygood AlexWaygood removed their request for review April 28, 2026 14:06
@brettcannon brettcannon removed their request for review April 28, 2026 16:44
Comment thread Doc/c-api/slots.rst Outdated
encukou and others added 2 commits April 30, 2026 12:07
@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32482235 | 📁 Comparing 3404bba against main (a386a52)

  🔍 Preview build  

36 files changed · + 2 added · ± 34 modified

+ Added

± Modified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants